Skip to content

Api#51

Merged
elysableu merged 7 commits intomainfrom
api
Nov 25, 2025
Merged

Api#51
elysableu merged 7 commits intomainfrom
api

Conversation

@elysableu
Copy link
Contributor

@elysableu elysableu commented Nov 25, 2025

Type of Change

  • feature ⛲
  • bug fix 🐛
  • documentation update 📃
  • styling 🎨
  • refactor 🧑‍💻
  • testing 🧪

Description

  • Update DataProvider to be able to request data from sg_BE
  • Add api.ts to handle api calls
  • Add JsonApiClient to handle data extraction
  • Updated config.ts with benign constants
  • Add updated props to components and passed to ProfileDetails component

Share the reason(s) for this pull request

What questions do you have/what do you want feedback on?

Related Tickets

Closes #38
Closes #39
Closes #48
Closes #50

Screenshots (if applicable):

Added Test?

  • Yes 🫡
  • No 🙅

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation and I have updated the documentation accordingly.
  • All tests (previous and new) pass 🥳

How to QA this change:

@elysableu elysableu merged commit e496608 into main Nov 25, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant